Skip to content

Conversation

@Zeophlite
Copy link
Owner

@Zeophlite Zeophlite commented Dec 2, 2025

Objective

Solution

  • Added ShortStandardMaterial to bevy_material , this is a subset of fields in bevy_pbr::StandardMaterial
  • Added MarkerMeshMaterial3d to bevy_material , this has a Handle to the ShortStandardMaterial
  • Added a system swap_marker_mesh_material_3d to bevy_pbr that:
    • queries for MarkerMeshMaterial3d
    • creates a StandardMaterial from the ShortStandardMaterial
    • removes the MarkerMeshMaterial3d
    • adds a MeshMaterial3d with the new StandardMaterial

Please bikeshed ShortStandardMaterial and MarkerMeshMaterial3d

Above removes the direct dependency, but adds an indirect one.

Alternative

I could move ShortStandardMaterial to bevy_gltf (rename it GltfMaterial ) , and then have a new crate bevy_gltf_render that depends on bevy_pbr , and somehow split load_node to add the MeshMaterial3d , but that would be a much larger change. I think this is simpler

See #6

Testing

  • cargo run --example load_gltf

@github-actions
Copy link

github-actions bot commented Dec 2, 2025

You added a new feature but didn't update the readme. Please run cargo run -p build-templated-pages -- update features to update it, and commit the file change.

@github-actions
Copy link

github-actions bot commented Dec 2, 2025

You added a new feature but didn't update the readme. Please run cargo run -p build-templated-pages -- update features to update it, and commit the file change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants